Add backlog, canceled, and duplicate task states - #1797
Conversation
There was a problem hiding this comment.
All reported issues were addressed across 15 files
Reply with feedback, questions, or to request a fix.
Fix all with cubic | Re-trigger cubic
cc0869b to
6cc2152
Compare
6b75a0b to
e3dca98
Compare
There was a problem hiding this comment.
All reported issues were addressed across 15 files (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Fix all with cubic | Re-trigger cubic
a879b59 to
24fc783
Compare
There was a problem hiding this comment.
All reported issues were addressed across 20 files (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Fix all with cubic | Re-trigger cubic
24fc783 to
80edff2
Compare
There was a problem hiding this comment.
1 issue found across 20 files (changes from recent commits).
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="pkg/coredata/migrations/20260828T133808Z.sql">
<violation number="1" location="pkg/coredata/migrations/20260828T133808Z.sql:21">
P1: During a rolling deployment, this migration makes new task states writable while target-branch instances may still read tasks. Those instances reject the new enum values in `TaskState.UnmarshalText`, so task queries can fail once a new state is used; deploy a tolerant reader before enabling these values or ensure old instances are fully drained before applying the migration.</violation>
</file>
Tip: Review your code locally with the cubic CLI to iterate faster.
Fix all with cubic | Re-trigger cubic
| -- OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE | ||
| -- SOFTWARE. | ||
|
|
||
| ALTER TYPE task_state ADD VALUE 'BACKLOG' BEFORE 'TODO'; |
There was a problem hiding this comment.
P1: During a rolling deployment, this migration makes new task states writable while target-branch instances may still read tasks. Those instances reject the new enum values in TaskState.UnmarshalText, so task queries can fail once a new state is used; deploy a tolerant reader before enabling these values or ensure old instances are fully drained before applying the migration.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At pkg/coredata/migrations/20260828T133808Z.sql, line 21:
<comment>During a rolling deployment, this migration makes new task states writable while target-branch instances may still read tasks. Those instances reject the new enum values in `TaskState.UnmarshalText`, so task queries can fail once a new state is used; deploy a tolerant reader before enabling these values or ensure old instances are fully drained before applying the migration.</comment>
<file context>
@@ -0,0 +1,23 @@
+-- OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+-- SOFTWARE.
+
+ALTER TYPE task_state ADD VALUE 'BACKLOG' BEFORE 'TODO';
+ALTER TYPE task_state ADD VALUE 'CANCELED';
+ALTER TYPE task_state ADD VALUE 'DUPLICATE';
</file context>
There was a problem hiding this comment.
Temporary breaking changes are fine here, it is an edge case on a non vital part of the app
4d9cea2 to
ccc46d5
Compare
There was a problem hiding this comment.
All reported issues were addressed across 22 files (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Fix all with cubic | Re-trigger cubic
ccc46d5 to
0afb894
Compare
There was a problem hiding this comment.
All reported issues were addressed across 23 files (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Fix all with cubic | Re-trigger cubic
e324a9d to
6c2c5e7
Compare
Signed-off-by: Sacha Al Himdani <sacha@probo.com>
6c2c5e7 to
34562ba
Compare
Summary by cubic
Adds
BACKLOG,CANCELED, andDUPLICATEtask states across storage, APIs, clients, and the UI. New tasks still default toTODOwhen no state is supplied, while callers can now create or update tasks in any state.Tests
+46-2Creates tasks in every supported state and confirms the default
TODOstate when none is given.Coredata
+33-1Adds the new enum values via migration and validates them alongside the existing states.
GraphQL API
+5-0Exposes the new states and accepts an optional
stateon task creation.MCP
+8-0Adds the new states and a
stateinput to the create-task tool schema and resolver.prb (CLI)
+31-1Adds a
--stateflag to task create, validates state on create and update, and generates flag help from the enum.Service
+8-1Passes an optional task state through validation and keeps
TODOas the fallback.App: console
+88-48Shows all six states in task forms and board columns, adds the state selector to create, and adds a Backlog-to-To-do quick move with English, French, and Dutch translations.
Package: ui
+66-1Adds icons, the exported
TaskStatetype, and Storybook stories for the new states.Package: n8n-node
+57-4Adds the new states to task create and update options.
Written for commit 34562ba. Summary will update on new commits.